Skip to content

Initial import of MultiRIOTCtrl#13

Open
miri64 wants to merge 1 commit intoRIOT-OS:masterfrom
miri64:multiriotctrl
Open

Initial import of MultiRIOTCtrl#13
miri64 wants to merge 1 commit intoRIOT-OS:masterfrom
miri64:multiriotctrl

Conversation

@miri64
Copy link
Member

@miri64 miri64 commented Jan 26, 2021

This allows to access multiple RIOTCtrl objects in a mapping-like manner:

>>> ctrl = MultiRIOTCtrl({'a': riotctrl.ctrl.RIOTCtrl(env={'BOARD': 'A'}),
...                       'b': riotctrl.ctrl.RIOTCtrl(env={'BOARD': 'B'})})
>>> ctrl.board()
{'a': 'A', 'b': 'B'}
>>> ctrl['a','b'].board()
{'a': 'A', 'b': 'B'}
>>> ctrl['a'].board()
'A'

The current base classes just iterate over the provided RIOTCtrls but the actual use case I am planning to use this for is to provide wrappers for

  • nodes with the same application to only compile the application once, and
  • iotlabcli + serial_aggregator.

@miri64 miri64 added the enhancement New feature or request label Jan 26, 2021
This was referenced Feb 16, 2021
Merged
Merged
@miri64
Copy link
Member Author

miri64 commented Mar 9, 2021

@aabadie @fjmolinas @leandrolanzieri any opinion on this?

@miri64 miri64 mentioned this pull request Mar 9, 2021
Merged
@fjmolinas
Copy link
Contributor

fjmolinas commented Sep 10, 2021

@aabadie @fjmolinas @leandrolanzieri any opinion on this?

Is this something that would allow easy integration into your iotlab_controller?

@miri64
Copy link
Member Author

miri64 commented Sep 10, 2021

Is this something that would allow easy integration into your iotlab_controller?

There are still some more ways to go for "easy" integration (such as serial_aggregator/tmux) support of ShellInteraction (or an extension / sibling of that), but in general, yes: this would simplify integration into iotlab_controller :-)

@MrKevinWeiss
Copy link
Contributor

MrKevinWeiss commented Jul 23, 2024

Is it worth pushing this?

@miri64
Copy link
Member Author

miri64 commented Jul 26, 2024

Could still make sense for experimentation, but I did not work on this in ages, so definitely would need some time to sort this out again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aabadie aabadie Awaiting requested review from aabadie

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss

@fjmolinas fjmolinas Awaiting requested review from fjmolinas

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@miri64 @fjmolinas @MrKevinWeiss